Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[QueryBuilder] Implement get_creation_statistics for SQLite backend #6763

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

rabbull
Copy link
Contributor

@rabbull rabbull commented Feb 19, 2025

This PR improves backend parity between two database backends by implementing get_creation_statistics for the SQLite backend. It achieves this by using an alternative method to implement date_trunc, which is available in PostgreSQL but not in SQLite.

Closes #6747.

@rabbull rabbull requested a review from GeigerJ2 February 19, 2025 10:19
@rabbull rabbull force-pushed the feature/sqlite_creation_stat branch from 14e5328 to 79e747d Compare February 19, 2025 10:27
Copy link

codecov bot commented Feb 19, 2025

Codecov Report

Attention: Patch coverage is 91.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 78.15%. Comparing base (f4c55f5) to head (4cf30a8).

Files with missing lines Patch % Lines
...rc/aiida/storage/psql_dos/orm/querybuilder/main.py 91.67% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6763      +/-   ##
==========================================
+ Coverage   78.15%   78.15%   +0.01%     
==========================================
  Files         564      564              
  Lines       42643    42654      +11     
==========================================
+ Hits        33322    33332      +10     
- Misses       9321     9322       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@GeigerJ2 GeigerJ2 changed the title [QueryBuilder] Implement get_creation_statastics for SQLite backend [QueryBuilder] Implement get_creation_statistics for SQLite backend Feb 20, 2025
GeigerJ2
GeigerJ2 previously approved these changes Feb 20, 2025
Copy link
Contributor

@GeigerJ2 GeigerJ2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neat and clean! Thanks!

@GeigerJ2 GeigerJ2 self-requested a review February 20, 2025 14:12
@GeigerJ2 GeigerJ2 dismissed their stale review February 20, 2025 14:13

Realized issue after approving

@rabbull
Copy link
Contributor Author

rabbull commented Feb 26, 2025

Hi @GeigerJ2, this PR is ready for another review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement get_creation_statistics for QB with SQLite backend
2 participants